-
Notifications
You must be signed in to change notification settings - Fork 1.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Workspace completion #5714
Workspace completion #5714
Conversation
✅ Deploy Preview for remixproject ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
}); | ||
|
||
// listen for file changes | ||
this.props.on('fileManager', 'fileRemoved', (path) => { this.indexed.isIndexed = false; }); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you already have this line 54
async getContextFiles(prompt) { | ||
try { | ||
if (!this.indexed.isIndexed) { | ||
this.indexWorkspace(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
do you need await
here?
); | ||
}; | ||
|
||
export default DynamicButtons; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
where is this used?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That slipped in from other work (adding context in chat)
package.json
Outdated
@@ -153,6 +153,7 @@ | |||
"express-ws": "^5.0.2", | |||
"file-path-filter": "^3.0.2", | |||
"file-saver": "^2.0.5", | |||
"flexsearch": "^0.7.43", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
do you need that dependency?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actually no, these are the bunch of alternative I tried out
…mix-project into workspace_completion
The user workspace is indexed in a regular time period. More context file is used for creating suggestions
For providing more From now on:
- Local file imports are resolved and added as context
- Relevant files on the workspace level are used as context to provide accurate suggestions